Skip to content

Update GroqAudioModels with new model names#5538

Open
vedevpatel wants to merge 1 commit intolivekit:mainfrom
vedevpatel:patch-2
Open

Update GroqAudioModels with new model names#5538
vedevpatel wants to merge 1 commit intolivekit:mainfrom
vedevpatel:patch-2

Conversation

@vedevpatel
Copy link
Copy Markdown
Contributor

Removing the deprecated 'distil-whisper-large-v3-en' model from the OpenAI list.

Removing the deprecated 'distil-whisper-large-v3-en' model from the OpenAI list.
Copilot AI review requested due to automatic review settings April 23, 2026 19:08
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the OpenAI plugin’s Groq STT model type list by removing a deprecated Groq Whisper model name.

Changes:

  • Remove "distil-whisper-large-v3-en" from GroqAudioModels in the OpenAI plugin models module.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 143 to 145
GroqAudioModels = Literal[
"whisper-large-v3", "distil-whisper-large-v3-en", "whisper-large-v3-turbo"
"whisper-large-v3", "whisper-large-v3-turbo"
]
Copy link

Copilot AI Apr 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR description mentions removing the deprecated model from the “OpenAI list”, but the actual change is to GroqAudioModels (Groq STT models) inside the OpenAI plugin. Consider updating the PR title/description to explicitly mention the Groq STT adapter to avoid confusion for reviewers/users (especially since the Groq plugin still lists this model).

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants